gtk_entry_completion_set_text_column: reuse old renderer

gtk_entry_completion_set_text_column() always added a cell renderer,
regardless of whether there was an existing one already installed.  This
patch reuses an old renderer if it exists, but only if it was added by a
previous call to this function.

To avoid conflicts, all renderers that were added manually are removed
when calling this function. Also, the renderer added by this function is
removed when manually adding new renderers. This effectively gives
GtkEntryCompletion two modes (managed and manual cell renderers) and
allows seamless switching between the two.

This is a minor API break. However, this shouldn't be an issue in
practice as applications couldn't call set_text_column() more than once
because of this bug. Also, it is unlikely that many applications mix
set_text_column() and custom cell renderers. The interaction between the
two modes was erratic and not documented well.

https://bugzilla.gnome.org/show_bug.cgi?id=635499

Fix "make install"

These files were listed twice

reftests: Add missing reftests to the test runner

gtksettings: Allow setting the GTK+ theme with an envvar

To test with Raleigh much easier.

frame: Remove a couple strange casts...

Win32 : fixes Bug 679019 (Popup menu misplaced in dual monitor)

GtkEntryCompletion: call setter for "text-column"

Call gtk_entry_completion_set_text_column() when setting the
"text-column" property directly.

The completion appeared empty when setting "text-column" directly (for
example from a GtkBuilder file), because the setter creates and adds the
GtkCellRendererText.

https://bugzilla.gnome.org/show_bug.cgi?id=710533

stack: Fix the slide animation if the child has a non-zero allocation position

This can happen if the child has a margin, for instance.

stack: Scope some variables so that the code is clearer

... that the allocation magic here is for the transition of the
last visible child that's sliding out.

gtkoverlay: Remove get_preferred_width / get_preferred_height implementation

GtkBin now handles this for us.

Win32: Honor the geometry when GDK_HINT_MAX_SIZE is set.

The MINMAXINFO struct was being populated based upon geometry hints when
GDK_HINT_MAX_SIZE flag was enabled, then promptly having its values blown
away with default values.

https://bugzilla.gnome.org/show_bug.cgi?id=711110

gtkbox: Draw backgrounds and borders

broadway-server: Fix signedness issue

Broadway : Replaced be32toh() with win32-compatible ntohl()

Make treeview cursor handling a bit more robust

Clearing the model in a GtkTreeSelection::changed handler was
crashing. While it is not a 100% kosher thing to do, lets be
more robust here.

Revert "wayland: Support always-on-top / sticky windows"

This reverts commit b3cffb85f39183a83a99e7517191e46bcd744749.

Pushed by accident.

Use g_return_val_if_fail where return value needed.

https://bugzilla.gnome.org/show_bug.cgi?id=710987

wayland: Support always-on-top / sticky windows

Use the new gtk-shell APIs available in mutter to add support for this.

https://bugzilla.gnome.org/show_bug.cgi?id=710056

wayland: Always attach null surfaces on hide

Destroying the surface isn't really appropriate, as the GtkWindow
is still realized and we won't necessarily know how to reconstruct it.

wayland: Remove a useless else-if

We already check for impl->surface as the first branch we do.

wayland: Make the code here a bit clearer

By grouping the destroy and field clear together.

wayland: Ensure we keep our internal state when hiding a window with no surface

We need to keep track of the mapped variable here.

gtkdialog: Allow specifying response IDs by nick in <action-widgets>

This makes it a lot more convenient for developers, as they don't
have to look up the numeric value of response IDs.

Bump version

Fix up accessibility tests

The output had gotten debug spew in it.

Update the release notes

Remove debug spew

GtkWidget: Fix clipping to large subwindows

_gtk_widget_draw_internal() was clipping by passing the subwindow
sizes as a path to cairo_clip(). This was breaking for windows
larger than 23 bits in width/height, due to cairo using fixed point
(24.8) for the path coordinates.

We fix this by pre-clipping the subwindow region to the existing
cairo clip region in the full 32bit gdkwindow precision. This fixes
the GooCanvas Large Items test.

https://bugzilla.gnome.org/show_bug.cgi?id=710958

Updates

Revert "dialog: Fix docs for GtkDialog"

This reverts commit e4bb02f8ae698facc204c616108cc8f4c3b579bc.

Responses don't have to be negative.

Update expected results for some a11y tests

These are affected by recent changes to stock icons (adding
accessible names) and spinner sizing.

Respect deprecated button images setting

I have been convinced that it is a bad idea to change the behaviour
at the same time as deprecating it, so go back to respecting the
Gtk/ButtonImages xsetting in buttons created with
gtk_button_new_from_stock() when it is set.

The setting as well as the function are still deprecated, and the
default value of the setting will remain FALSE.

Respect deprecated menu images setting

I have been convinced that it is a bad idea to change the behaviour
at the same time as deprecating it, so go back to respecting the
Gtk/MenuImages xsetting in GtkImageMenuItem when it is set.

The setting as well as the widget are still deprecated, and the
default value of the setting will remain FALSE.

Fix key "Page_Down" identified as "Next"

Since 49c4ad2f the order between "Page_Down" & "Next" had been changed (due to
sorting) and this resulted in the Page Down key being identified as "Next"
instead of "Page_Down"

"Page_Up" wasn't affected, since it comes before "Prior"

Signed-off-by: Olivier Brunel <jjk@jjacky.com>
https://bugzilla.gnome.org/show_bug.cgi?id=710411

Revert "GtkPlacesSidebar: support switching locations for XDS drag types"

This reverts commit 588ffa8c3241eff2f8e2d99b911a6bcb539347bd.

Revert "GtkPlacesSidebar: support open locations when dragging text"

This reverts commit 1b839d4b72a2cedb2bb633b5acd57239860693f4.

Revert "GtkPlacesSidebar: use gtk api to add dnd uri targets"

This reverts commit 7fa27dff259ad79d5a4364d7eaa6b63150082af3.

GtkPlacesSidebar: use gtk api to add dnd uri targets

Instead of hardcoding it in the dnd drop targets list.

Part of bug 707679

GtkPlacesSidebar: support open locations when dragging text

Make GtkPlacesSidebar also open locations when hovered by
text dnd targets.

Part of bug 707679

GtkPlacesSidebar: support switching locations for XDS drag types

Make GtkPlacesSidebar also change locations when hovered
by XDS dnd targets (eg. dragging a file from file-roller).

Part of bug 707679

dialog: Fix docs for GtkDialog

Responses need to be negative

GtkScrolledWindow: Disconnect from frame clock properly

The tick callback IDs from GtkWidget aren't timeouts, so
use the correct function to disconnect from them.

Spotted by Benjamin Otte <otte@redhat.com>

https://bugzilla.gnome.org/show_bug.cgi?id=710666

GtkWidget: Avoid lingering clock frame updates

For some widgets, like GtkTreeView, which setup a clock frame
update during realize, it was possible to call
gdk_frame_clock_begin_updating() twice, but only ever disconnecting
from it once. This happens because the realized flag is set at an
unpredictable time by the GtkWidget's realize implementation.

Keep the signal handler ID from us connecting to the "update" signal
to avoid connecting to it twice.

This fixes high wake-up count from any application using GtkTreeView,
even idle ones.

https://bugzilla.gnome.org/show_bug.cgi?id=710666

all: Add names to timeouts

Add names to every timeout we setup, so it's easier to track their
usage, and debug possible misbehaviour.

https://bugzilla.gnome.org/show_bug.cgi?id=710651

Fix a typo

Add missing symbol to docs

Bump GLib req to 2.39

Needed for GNotification.

Add some (silly) GNotification example

Bloatpad can now notify you.

gdkwindow-x11: Don't set WM_TAKE_FOCUS if accept_focus is FALSE

Otherwise, we'll be in the "Globally Active" mode instead of the
"No Input" mode.

https://bugzilla.gnome.org/show_bug.cgi?id=710389

Icon theme: Hardcode fallback through the 'gnome' icon theme

This is what we used to get through the Net/FallbackIcontheme
setting. Nobody has ever set this setting to a different value,
and people have come to rely on GTK+ applications getting their
icons this way.

flowbox: Always return a NULL iter when we are at the end

The objective is simplify the semantics of the code so that we don't
need to check for "(iter != NULL && !g_sequence_iter_is_end (iter))"
in the callers.

flowbox: Be careful when the container is empty

Otherwise, showing an empty GtkFlowBox will lead to:
GLib-CRITICAL **: g_sequence_get: assertion '!is_end (iter)' failed

gtktoolbar: Make the icon-size property actually an enum

As this is what it's interpreted as.

Updated Russian translation

stackswitcher: Only show a button if the stack child is visible

gtkflowbox: Fix warning when realizing

set_background needs the widget's window to be set.

Add docs for new API

Add new symbols to the docs

Add an index for symbols added in 3.12

Fix since tags

Mark 3.12 api as such.

notebook: inform the theme whether show-borders was set

So that it can draw a border or not. If show-borders=true then
the notebook will have GTK_STYLE_CLASS_FRAME.

Remove superfluous g_file_test()

https://bugzilla.gnome.org/show_bug.cgi?id=168530
Signed-off-by: Timothy Arceri <t_arceri@yahoo.com.au>

Fix GtkToolButton possibly being empty

With style BOTH_HORIZ the label would not be selected (if not marked
"important") and when there's no icon, it would result in an empty button.

Signed-off-by: Olivier Brunel <jjk@jjacky.com>
https://bugzilla.gnome.org/show_bug.cgi?id=710134

GtkBuilder: Fall back to gtk_test_register_all_types if type lookup fails

This is important because the dlsym() approach can fail if gtk was
loaded with RTLD_LOCAL.

https://bugzilla.gnome.org/show_bug.cgi?id=710096

gdkevents.h: Mention _get_source_device()

It's too easy getting bitten by the ->device red herring, thinking
that it's the original input device the event originated from.

https://bugzilla.gnome.org/show_bug.cgi?id=709621

Fix an oversight

Followup for 3c41d4865e230f41a3f4cb5f72e438a0788c7bb3

Fall back to non-symbolic icons

When doing fallback for symbolic icons, we first shorten
the name at dashes while preserving the -symbolic suffix.
But after exhausting that, we should also try stripping
the suffix.

https://bugzilla.gnome.org/show_bug.cgi?id=708163

tooltip: Fix possible wrong placement

When showing a tooltip on the edge of a monitor, the tooltip could be wrongly
placed and be shown going from one monitor to the next.

This happened because the current_window wasn't set visible, and when it wasn't
the returned allocated size would be 1, hence wrong calculations.

Signed-off-by: Olivier Brunel <jjk@jjacky.com>
https://bugzilla.gnome.org/show_bug.cgi?id=698730

Fix a warning

The PangoWrapMode enum has identical layout to GtkWrapMode,
but using the two interchangably causes (justified) compiler
warnings.

https://bugzilla.gnome.org/show_bug.cgi?id=709697

Set an error when loading a GtkIconInfo fails

Failure to do so leads to assertion failures.
https://bugzilla.gnome.org/show_bug.cgi?id=710073

Wayland: avoid accidental export of internal symbols

Some symbols in the generated Wayland code were getting
decorated with WL_EXPORT, causing them to show up in the
libgdk exports. We don't want that.

https://bugzilla.gnome.org/show_bug.cgi?id=710141

Updated POTFILES.in

Wayland: fix a crash in opaque region handling

We may get a NULL region passed to the backend, which means
'nothing is opaque'. In that case, don't crash, but pass
the information on to the compositor.

http://bugzilla.gnome.org/show_bug.cgi?id=709854

bloatpad: test dynamic accels

...this stuff will be in the action description soon.

GtkApplication: a new approach to accels

Rework how accels are handled on GtkApplicationWindow.

Instead of having GtkApplication fill the GtkAccelMap which is then used
by GtkApplicationWindow to create a GtkAccelGroup filled with closures
that is then associated with the window, do it directly.

GtkApplication now keeps a list of accels and their actions.
Accelerators on a GtkApplicationWindow ask GtkApplication to execute the
appropriate action.

This saves a fair bit of complexity and memory use (due to not having to
create all those closures and accelmap entries).  The new approach also
supports multiple accels per action (although there is not yet a public
API for it).

This patch (and the ones before) Reviewed and ACK'd by Matthias Clasen.

GtkMenuTrackerItem: add support for dynamic accels

Add support for pulling the primary accel out of the GtkActionMuxer.

With this change, it is no longer necessary to have the accel=''
attribute hardcoded onto each menu item (and, in fact, it should be left
off if you intend to have support for dynamic accelerator changing).

Specifying accel='' is a good way to force an accelerator not to be
displayed on a menu item.

GtkMenuTrackerItem: use "action and target" format

Store "action and target" format inside each GtkMenuTrackerItem.  This
makes action invocation more efficient (no hash table lookups or
allocations) and slightly simplifies handling of action namespace.

More importantly, this will be used when we start to get accels from
GtkActionMuxer.

GtkActionMuxer: store primary accels

Reuse the existing infrastructure in GtkActionMuxer for propagation of
accelerator information: in particular, what accel label ought to appear
on menu items for a particular action and target.

This is a good idea because we want accels to travel along with the
actions that they're tied to and reusing GtkActionMuxer will allow us to
do that without creating another hierarchy of a different class for the
sole purpose of filling in accel labels on menu items.

Doing it this ways also allows those who copy/paste GtkActionMuxer to
insert the accels for themselves.

Add a new method on the GtkActionObserver interface to report changes.

This patch introduces a new concept: "action and target" notation for
actions.  This format looks like so:

  "'target'|app.action"

or for non-targeted actions:

  "|app.action"

and it is used over a number of possible alternative formats for some
good reasons:

 - it's very easy to get a nul-terminated action name out of this format
   when we need it, by using strrchr('|') + 1

 - we can also get the target out of it using g_variant_parse() because
   this function takes a pointer to a 'limit' character that is not
   parsed past: we use the '|' for this

 - it's extremely easy to hash on this format (just use a normal string
   hash) vs. attempting to hash on a string plus a GVariant

A close contender was to use detailed action strings here, but these are
not used for two reasons:

 - it's not possible to easily get the action name or target out of the
   strings without more work than the "action and target" format
   requires

 - we still intend to use detailed action strings on API (since they are
   a lot nicer to look at) but detailed action strings can be given in
   non-canonical forms (eg: 'foo::bar' and 'foo("bar")' are equivalent)
   so we'd have to go through a normalisation step anyway.  Since we're
   doing that already, we may as well convert to a more convenient
   internal format.

This new "action and target" format is going to start appearing in a lot
more places as action descriptions are introduced.

I suspect that nobody is using '|' in their action names, but in case I
am proven wrong, we can always switch to using something more exotic as
a separator character (such as '\x01' or '\xff' or the like).

GtkWindow: change muxer setup with application

Previously, GtkWindow would add the "app" action group to its own
toplevel muxer.

Change the setup so that GtkApplication creates the toplevel muxer and
adds itself to it as "app".  Use this muxer as the parent muxer of any
GtkWindow associated with the application.

This saves a small amount of memory and will allow for accels to be
propagated from the application through to all of the windows.

GtkBuilder: add GtkApplication

Add a GtkApplication (private) field to GtkBuilder

Fix a crash in icon handling

The load_error was freed in two places.
Fix based on a patch in
https://bugzilla.gnome.org/show_bug.cgi?id=709967

Updated Spanish translation

Updated Indonesian translation

Restore accessible names for image-only buttons

With the stock system being deprecated now, we should provide
meaningful accessible names for buttons that are constructed
from icon names or GIcons. This commit reuses the existing
translations.

It is possible that some common icon names are not covered
here because they were not present as stock items. These can
be added to the table later.

[l10n] Updated Catalan (Valencian) translation

[l10n] Update Catalan translation

Updated translation for Afrikaans (af)

Bug 709939 - monitor reconfiguration while GtkOffscreenWindows

exist triggers crash on quartz. Special case the position update
to ignore the off-screen windows.

Quell a compiler warning

NULL check on default keymap

Ensure that Quartz's default keymap is not accessed before it is
created.

Fix infinite loop in gdkevents.c _gdk_event_queue_find_first

Ensure src and dest are GdkWindows in _gdk_sythesize_crossing_events

Discovered via a crash because b's (dest's) toplevel was NULL;
ensuring that the dest is actually a GdkWindow or setting b to NULL
prevents that path from being taken.

Fix double-free introduced in f86dc163

spinner: don't expand to fill the allocated space

Spinners are essentially animated symbolic icons. We don't really
want them being rendered to arbitrary sizes. Also the current
technique for rendering the activity is not suitable for larger
sizes. Until we have a better technique we should limit the size
to the MENU icon size or exactly twice that size.

listbox: Fix keynav_failed() parameter in move_cursor()

The 'direction' parameter to gtk_widget_keynav_failed() is based on
gtk_list_box_move_cursor()'s 'count' parameter. However if the passed
in movement is GTK_MOVEMENT_DISPLAY_LINES, 'count' is modified by
the keynav handling and will always be 0. To avoid messing up the
'direction' parameter, use a local variable for keynav handling and
leave 'count' untouched.

https://bugzilla.gnome.org/show_bug.cgi?id=709687

GMenu docs: Tiny grammar fix

L10N: Updated Persian translations

Suppress cursor theme unsupported warning when there's no theme.

Implement gtk_drag_begin_with_coordinates () for quartz.

Suppress frequent device assertions

Because send crossing event is usually called with a NULL source_device in quartz.